Skip to content

Make error code registration work again. #19624 - #21289

Merged
bors merged 7 commits into
rust-lang:masterfrom
brson:errorcodes
Jan 22, 2015
Merged

bors merged 7 commits into
rust-lang:masterfrom
brson:errorcodes

Conversation

@brson

@brson brson commented Jan 17, 2015

Copy link
Copy Markdown
Contributor

This does the bare minimum to make registration of error codes work again. After this patch, every call to span_err! with an error code gets that error code validated against a list in that crate and a new tidy script errorck.py validates that no error codes are duplicated globally.

There are further improvements to be made yet, detailed in #19624.

r? @nikomatsakis

@brson
brson force-pushed the errorcodes branch 3 times, most recently from d910a17 to ac6d6d9 Compare January 17, 2015 19:51
@brson

brson commented Jan 17, 2015

Copy link
Copy Markdown
Contributor Author

The tidy script now also prints the highest error code.

@brson
brson force-pushed the errorcodes branch 3 times, most recently from deacf5f to ab410cd Compare January 17, 2015 22:09
@brson
brson force-pushed the errorcodes branch 2 times, most recently from 2df3c3d to 56f77ec Compare January 19, 2015 00:58
@alexcrichton

Copy link
Copy Markdown
Member

@bors: r+ 7528c2c

@brson

brson commented Jan 20, 2015

Copy link
Copy Markdown
Contributor Author

@bors: r=alexcrichton 9a4401f

@brson

brson commented Jan 20, 2015

Copy link
Copy Markdown
Contributor Author

@bors: r=alexcrichton 876b266

@bors

bors commented Jan 21, 2015

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 876b266 with merge 0a1e3ef...

alexcrichton added a commit to alexcrichton/rust that referenced this pull request Jan 21, 2015
This does the bare minimum to make registration of error codes work again. After this patch, every call to `span_err!` with an error code gets that error code validated against a list in that crate and a new tidy script `errorck.py` validates that no error codes are duplicated globally.

There are further improvements to be made yet, detailed in rust-lang#19624.

r? @nikomatsakis
@bors

bors commented Jan 21, 2015

Copy link
Copy Markdown
Collaborator

💔 Test failed - auto-linux-64-nopt-t

@alexcrichton

Copy link
Copy Markdown
Member

@bors: retry

@bors

bors commented Jan 21, 2015

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 876b266 with merge d6a75a5...

@bors

bors commented Jan 21, 2015

Copy link
Copy Markdown
Collaborator

💔 Test failed - auto-linux-64-nopt-t

@alexcrichton

Copy link
Copy Markdown
Member

@bors: retry

1 similar comment
@alexcrichton

Copy link
Copy Markdown
Member

@bors: retry

@bors
bors merged commit 876b266 into rust-lang:master Jan 22, 2015
lnicola pushed a commit to lnicola/rust that referenced this pull request Dec 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants